Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove unused function. #1312

Merged
merged 1 commit into from
Jul 3, 2016
Merged

Remove unused function. #1312

merged 1 commit into from
Jul 3, 2016

Conversation

gusty
Copy link
Contributor

@gusty gusty commented Jul 2, 2016

This function is not called anywhere in the code.

This function is not called anywhere in the code.
@msftclas
Copy link

msftclas commented Jul 2, 2016

Hi @gmpl, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@@ -1922,11 +1922,6 @@ and CanMemberSigsMatchUpToCheck
//-------------------------------------------------------------------------


and private DefinitelyEquiv (csenv:ConstraintSolverEnv) isConstraint calledArg (CallerArg(callerArgTy,m,_,_) as callerArg) =
let calledArgTy = AdjustCalledArgType csenv.InfoReader isConstraint calledArg callerArg
if not (typeEquiv csenv.g calledArgTy callerArgTy) then ErrorD(Error(FSComp.SR.csArgumentTypesDoNotMatch(),m)) else
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is csArgumentTypesDoNotMatch used anywhere else?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, in ArgsEquivInsideUndo for instance.

@dsyme dsyme merged commit a4c3d40 into dotnet:master Jul 3, 2016
@dsyme
Copy link
Contributor

dsyme commented Jul 3, 2016

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants